Add OutputScope for output message tracing with parent span linking by Copilot · Pull Request #164 · microsoft/Agent365-python

Copilot · 2026-02-06T15:15:45Z

Extracts output span functionality from PR #113. Adds OutputScope for tracing agent output messages with W3C Trace Context parent span linking.

New files

spans_scopes/output_scope.py - OpenTelemetry scope for output messages
models/response.py - Response dataclass

Base scope changes

Added parent_id parameter to OpenTelemetryScope.__init__
Parent context parsed from W3C format and passed at span creation (OTel Python has no post-creation parent setter)

Utils refactoring

parse_parent_id_to_context() moved to utils.py
Added validate_trace_id(), validate_span_id() with hex validation

Behavior

record_output_messages() appends to accumulated list (not replace)

response = Response(messages=["Hello"])
parent_id = "00-1234567890abcdef1234567890abcdef-abcdefabcdef1234-01"

with OutputScope.start(agent_details, tenant_details, response, parent_id=parent_id) as scope:
    scope.record_output_messages(["More"])  # Result: ["Hello", "More"]

Tests

Reduced from 10 to 4 focused tests using real spans via InMemorySpanExporter.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

il-1234567812345678123456781234567.8.tenant.api.powerplatform.com
- Triggering command: /home/REDACTED/work/Agent365-python/Agent365-python/.venv/bin/pytest pytest tests/observability/core/ -v --tb=short -m not integration (dns block)
il-testtenan.t.tenant.api.powerplatform.com
- Triggering command: /home/REDACTED/work/Agent365-python/Agent365-python/.venv/bin/pytest pytest tests/observability/core/ -v --tb=short -m not integration (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Configure Actions setup steps to set up my environment, which run before the firewall is enabled
Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Add Response model class for agent execution response details - Add spans_scopes directory with OutputScope implementation - Add unit tests for OutputScope functionality Co-authored-by: nikhilNava <[email protected]>

Co-authored-by: nikhilNava <[email protected]>

Similar to the .NET ExecuteToolScope, the OutputScope now accepts an optional parent_id parameter to link spans to upstream operations. Co-authored-by: nikhilNava <[email protected]>

Similar to the .NET implementation, the base OpenTelemetryScope class now accepts an optional parent_id parameter that sets the CUSTOM_PARENT_SPAN_ID_KEY attribute on the span. This allows all scope subclasses to support parent span linking consistently. Co-authored-by: nikhilNava <[email protected]>

Changed the behavior of record_output_messages to append new messages to the accumulated list rather than replacing all messages. This allows collecting output messages over multiple calls during the scope's lifetime. Co-authored-by: nikhilNava <[email protected]>

Changed the parent_id implementation to properly link spans by parsing the W3C Trace Context format parent ID and using it to create a proper span context. This aligns with the .NET implementation which calls `activity?.SetParentId(parentId!)`. Added validation for W3C Trace Context version and trace_id/span_id lengths. Co-authored-by: nikhilNava <[email protected]>

Refactored W3C Trace Context parsing: - Added W3C Trace Context validation constants - Added validate_w3c_trace_context_version() helper - Added validate_trace_id() helper with hex validation - Added validate_span_id() helper with hex validation - Moved parse_parent_id_to_context() from opentelemetry_scope.py Co-authored-by: nikhilNava <[email protected]>

…similar ones Reduced from 10 tests to 4 focused tests: - test_output_scope_creates_span_with_messages (merged span name and messages tests) - test_record_output_messages_appends (merged multiple append tests) - test_output_scope_with_parent_id (parent linking) - test_output_scope_dispose (manual dispose) All tests use real spans via InMemorySpanExporter (no mocks). Co-authored-by: nikhilNava <[email protected]>

Copilot

Pull request overview

Adds a dedicated OutputScope to the observability core so agent output messages can be traced as their own OpenTelemetry span, with optional W3C Trace Context (traceparent) parent span linking.

Changes:

Introduces OutputScope for recording/appending output messages onto a span attribute.
Adds Response dataclass and extends OpenTelemetryScope to accept parent_id, parsing it into an OTel parent context at span creation time.
Refactors W3C parent parsing into utils.py, adding basic hex/id validation helpers, and adds focused unit tests using InMemorySpanExporter.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 9 comments.

Show a summary per file

File	Description
`tests/observability/core/test_output_scope.py`	New unit tests covering span creation, append behavior, and parent linking for `OutputScope`.
`.../observability/core/utils.py`	Adds W3C traceparent parsing + validation helpers and returns an OTel context containing a remote parent span.
`.../observability/core/spans_scopes/output_scope.py`	New `OutputScope` implementation that sets/appends `gen_ai.output.messages`.
`.../observability/core/spans_scopes/__init__.py`	Introduces the package init for `spans_scopes`.
`.../observability/core/opentelemetry_scope.py`	Adds `parent_id` support by using parsed parent context when starting spans.
`.../observability/core/models/response.py`	New `Response` dataclass for output message payload.

...s/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/utils.py

tests/observability/core/test_output_scope.py

...365-observability-core/microsoft_agents_a365/observability/core/spans_scopes/output_scope.py

...t-agents-a365-observability-core/microsoft_agents_a365/observability/core/models/response.py

...365-observability-core/microsoft_agents_a365/observability/core/spans_scopes/output_scope.py

...ents-a365-observability-core/microsoft_agents_a365/observability/core/opentelemetry_scope.py

tests/observability/core/test_output_scope.py

...t-agents-a365-observability-core/microsoft_agents_a365/observability/core/models/response.py

Initial plan

765af45

Copilot AI assigned Copilot and nikhilNava Feb 6, 2026

Copilot started work on behalf of nikhilNava February 6, 2026 15:16 View session

Copilot AI and others added 2 commits February 6, 2026 15:19

Add OutputScope for output message tracing

482cb38

- Add Response model class for agent execution response details - Add spans_scopes directory with OutputScope implementation - Add unit tests for OutputScope functionality Co-authored-by: nikhilNava <[email protected]>

Improve Response.messages field documentation

53bd58d

Co-authored-by: nikhilNava <[email protected]>

Copilot AI changed the title ~~[WIP] Extract changes to output span from existing PR~~ Add OutputScope for output message tracing Feb 6, 2026

Copilot AI requested a review from nikhilNava February 6, 2026 15:21

Copilot finished work on behalf of nikhilNava February 6, 2026 15:21

Copilot started work on behalf of nikhilNava February 6, 2026 15:31 View session

Add parent_id parameter to OutputScope

9ddca17

Similar to the .NET ExecuteToolScope, the OutputScope now accepts an optional parent_id parameter to link spans to upstream operations. Co-authored-by: nikhilNava <[email protected]>

Copilot AI changed the title ~~Add OutputScope for output message tracing~~ Add OutputScope with parent_id for output message tracing Feb 6, 2026

Copilot finished work on behalf of nikhilNava February 6, 2026 15:34

Copilot started work on behalf of nikhilNava February 6, 2026 15:38 View session

Copilot AI changed the title ~~Add OutputScope with parent_id for output message tracing~~ Add OutputScope with parent_id support on base OpenTelemetryScope Feb 6, 2026

Copilot finished work on behalf of nikhilNava February 6, 2026 15:42

Copilot started work on behalf of nikhilNava February 6, 2026 15:49 View session

Copilot AI changed the title ~~Add OutputScope with parent_id support on base OpenTelemetryScope~~ Add OutputScope for output message tracing with parent_id support Feb 6, 2026

Copilot finished work on behalf of nikhilNava February 6, 2026 15:53

Copilot started work on behalf of nikhilNava February 6, 2026 16:33 View session

Copilot AI changed the title ~~Add OutputScope for output message tracing with parent_id support~~ Add OutputScope for output message tracing with proper parent span linking Feb 6, 2026

Copilot finished work on behalf of nikhilNava February 6, 2026 16:39

Copilot started work on behalf of nikhilNava February 6, 2026 16:47 View session

Copilot finished work on behalf of nikhilNava February 6, 2026 16:50

Copilot started work on behalf of nikhilNava February 6, 2026 16:59 View session

Copilot AI changed the title ~~Add OutputScope for output message tracing with proper parent span linking~~ Add OutputScope for output message tracing with parent span linking Feb 6, 2026

Copilot finished work on behalf of nikhilNava February 6, 2026 17:02

Copilot started work on behalf of nikhilNava February 6, 2026 17:22 View session

Copilot AI changed the title ~~Add OutputScope for output message tracing with parent span linking~~ Add OutputScope for output message tracing with W3C parent span linking Feb 6, 2026

Copilot finished work on behalf of nikhilNava February 6, 2026 17:27

Copilot started work on behalf of nikhilNava February 6, 2026 18:20 View session

Copilot AI changed the title ~~Add OutputScope for output message tracing with W3C parent span linking~~ Add OutputScope for output message tracing with parent span linking Feb 6, 2026

Copilot finished work on behalf of nikhilNava February 6, 2026 18:24

Merge branch 'main' into copilot/extract-output-span-changes

d36e3e8

nikhilNava marked this pull request as ready for review February 9, 2026 13:31

nikhilNava requested a review from a team as a code owner February 9, 2026 13:31

Copilot AI review requested due to automatic review settings February 9, 2026 13:31

Copilot started reviewing on behalf of nikhilNava February 9, 2026 13:32 View session

Copilot AI reviewed Feb 9, 2026

View reviewed changes

juliomenendez previously approved these changes Feb 9, 2026

View reviewed changes

...t-agents-a365-observability-core/microsoft_agents_a365/observability/core/models/response.py Outdated Show resolved Hide resolved

address PR comments

3901fa4

nikhilNava dismissed juliomenendez’s stale review via 3901fa4 February 9, 2026 14:38

nikhilNava enabled auto-merge (squash) February 9, 2026 14:39

juliomenendez approved these changes Feb 9, 2026

View reviewed changes

thanikaReddy approved these changes Feb 9, 2026

View reviewed changes

threddy approved these changes Feb 9, 2026

View reviewed changes

nikhilNava merged commit eaa2136 into main Feb 9, 2026
14 checks passed

nikhilNava deleted the copilot/extract-output-span-changes branch February 9, 2026 17:39

Copilot AI mentioned this pull request Feb 11, 2026

Add parent_id support to ExecuteToolScope and InferenceScope #177

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add OutputScope for output message tracing with parent span linking#164

Add OutputScope for output message tracing with parent span linking#164
nikhilNava merged 11 commits intomainfrom
copilot/extract-output-span-changes

Copilot AI commented Feb 6, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

Copilot AI commented Feb 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

New files

Base scope changes

Utils refactoring

Behavior

Tests

I tried to connect to the following addresses, but was blocked by firewall rules:

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Copilot AI commented Feb 6, 2026 •

edited

Loading